Skip to content

Conversation

harishreedharan
Copy link

We use the Async Kudu Client (and I have used the Async HBase client before), and we have issues testing this in unit tests because the Deferred class is final. To mock it, we have to jump through hoops with power mock. I don't see a reason for this class to be final, so just removing the final keyword.

We use the Async Kudu Client (and I have used the Async HBase client before), and we have issues testing this in unit tests because the Deferred class is final. To mock it, we have to jump through hoops with power mock. I don't see a reason for this class to be final, so just removing the final keyword.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant